Skip to content

Rename suppression "state" property to "status" - #1667

Closed
ykim-r7 wants to merge 1 commit into
github:mainfrom
ykim-r7:patch-1
Closed

ykim-r7 wants to merge 1 commit into
github:mainfrom
ykim-r7:patch-1

Conversation

@ykim-r7

@ykim-r7 ykim-r7 commented May 2, 2023 •

Copy link
Copy Markdown

This property is supposed to be called "status", per the SARIF spec: https://docs.oasis-open.org/sarif/sarif/v2.1.0/os/sarif-v2.1.0-os.html#_Toc34317736:~:text=to%20be%20ignored.-,3.35.3%20status%20property,-A%20suppression%20object

Merge / deployment checklist

  • Confirm this change is backwards compatible with existing workflows.
  • Confirm the readme has been updated if necessary.
  • Confirm the changelog has been updated if necessary.

This property is supposed to be called "status", per the SARIF spec.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant